Skip to content

[OldMongo FC-0004] Prepare tests for removing support for children in Old Mongo#31224

Merged
ormsbee merged 1 commit intoopenedx:masterfrom
raccoongang:sagirov/EDXOLDMNG-146-1-part
Oct 28, 2022
Merged

[OldMongo FC-0004] Prepare tests for removing support for children in Old Mongo#31224
ormsbee merged 1 commit intoopenedx:masterfrom
raccoongang:sagirov/EDXOLDMNG-146-1-part

Conversation

@UvgenGen
Copy link
Contributor

@UvgenGen UvgenGen commented Oct 28, 2022

Description

Prepare tests for removing support for children in Old Mongo (#31134)
Updated related tests

Useful information to include:
openedx/public-engineering#80

@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Oct 28, 2022
@openedx-webhooks
Copy link

openedx-webhooks commented Oct 28, 2022

Thanks for the pull request, @UvgenGen! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

@UvgenGen UvgenGen force-pushed the sagirov/EDXOLDMNG-146-1-part branch 2 times, most recently from f8a046e to 9a45745 Compare October 28, 2022 12:33
@UvgenGen UvgenGen force-pushed the sagirov/EDXOLDMNG-146-1-part branch from 9a45745 to 877573c Compare October 28, 2022 12:53
@UvgenGen UvgenGen mentioned this pull request Oct 28, 2022
Merged
ormsbee
ormsbee approved these changes Oct 28, 2022
View reviewed changes
Copy link
Contributor

@ormsbee ormsbee left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. @jristau1984: I'm planning to merge this shortly, given that it only affects tests.

@ormsbee ormsbee merged commit 8f88422 into openedx:master Oct 28, 2022
@openedx-webhooks
Copy link

openedx-webhooks commented Oct 28, 2022

@UvgenGen 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

@edx-pipeline-bot
Copy link
Contributor

edx-pipeline-bot commented Oct 28, 2022

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

edx-pipeline-bot commented Oct 28, 2022

EdX Release Notice: This PR has been deployed to the production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ormsbee ormsbee ormsbee approved these changes

Assignees

No one assigned

Labels

open-source-contribution PR author is not from Axim or 2U

Projects

Axim Engineering Tasks
Archived in project
Contributions
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@UvgenGen @openedx-webhooks @edx-pipeline-bot @ormsbee